Description

• Authenticated Sysinfo

currently previously the the internal /internal/sysinfo /internal/sysinfo-download routes are not authenticated

I think AUTO's intention was that so the same API can be reused

but this has lead to a security issue in the past and could lead to more in the future if not already
so to be safe it's better that these routes require authenticated

this PR changes how sysinfo tabs downloads sysinfo, using javascript button clicks and gradio calls
(as opposed to browser to API calls)
and as an extra benefit there's now a progress effect on the HTML while retrieving sysinfo

the Sysinfo tab no longer no longer relies on the API
instead it's gradio button click event that generates sysinfo to a hidden text box
followed by JavaScript to download as file or open this info in new taba hidden textarea

this is overkill

normally the sysinfo should be in the realms of kilobytes so it should be totally fine to store it in textarea

I have tested that it works for even text size of 256MB
but as it is technically possible for the size to be extremely large (maybe there are lots of exceptions or something)
I implemented a logic that if the file is greater than 1MB, it would writes to tmp/sysinfo.json and then use file= route to retrieve it

for backwards compatibility the existing /internal/sysinfo /internal/sysinfo-download are moved to models.api
the behavior is unchanged except that it now would respect --api-auth and would only be created if --api is enabled

tested to work in Chrome and Firefox

additional change

• fix shadows name 'k' from outer scope

this line of code triggers a false error in pycharms inspector
I decided to add in because I'm already working on this file

Screenshots/videos:

firefox

false error

Checklist:

• I have read contributing wiki page
• I have performed a self-review of my own code
• My code follows the style guidelines
• My code passes tests